Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open AI panel when AI app generator starts generating #1989

Conversation

jurgenwerk
Copy link
Contributor

@jurgenwerk jurgenwerk commented Dec 27, 2024

This PR adjusts the following:

  • Removes experimentalAIEnabled flag - I believe this is not experimental anymore
  • Moves currentRoomId and aiAssistantOpen from component state to matrix and operator mode state services so that we can manipulate them from other parts of the app
  • Adds openRoom command
  • AI app generator will open the AI panel once it starts generating the app, allowing the user to observe the automatic prompts and generations from the AI bot

Here in this video you can see the AI panel opens when user clicks on "Let's get started":

Screen.Recording.2024-12-27.at.13.24.06.mov

Copy link

github-actions bot commented Dec 27, 2024

Host Test Results

    1 files  ±0      1 suites  ±0   20m 21s ⏱️ +10s
720 tests +2  718 ✔️ +2  2 💤 ±0  0 ±0 
725 runs  +2  723 ✔️ +2  2 💤 ±0  0 ±0 

Results for commit 23773c8. ± Comparison against base commit 71f2711.

♻️ This comment has been updated with latest results.

@@ -622,7 +623,7 @@ module('Acceptance | Commands tests', function (hooks) {
assert.dom('[data-test-card-url-bar-input]').hasValue(`${testCard}.json`);
await click('[data-test-submode-switcher] button');
await click('[data-test-boxel-menu-item-text="Interact"]');
await click('[data-test-open-ai-assistant]');
Copy link
Contributor Author

@jurgenwerk jurgenwerk Dec 27, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The behavior changed now that I moved the aiAssistantOpen from the component to the state service–previously, if the AI panel was open and the user toggled from interactive to code mode and then back to interactive, the AI panel was closed. But now, the AI panel remains open when going back to interactive mode. I think this behavior is actually more desirable, let me know if you think otherwise.

@jurgenwerk jurgenwerk requested a review from a team December 27, 2024 12:37
@jurgenwerk jurgenwerk marked this pull request as ready for review December 27, 2024 12:37
Copy link
Contributor

@lukemelia lukemelia left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Instead of introducing a new action on context.actions, we should introduce a new host command.

@jurgenwerk jurgenwerk requested review from lukemelia and a team December 30, 2024 11:13
@jurgenwerk
Copy link
Contributor Author

I added a new command for the purpose of opening an AI assistant room.

Copy link
Contributor

@lukemelia lukemelia left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice work

@jurgenwerk jurgenwerk merged commit 6717eb2 into main Dec 31, 2024
38 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants